-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added course purchases and auth system #324
Conversation
2c47eb0
to
2cab054
Compare
This is great, will review later today |
7597706
to
2bee295
Compare
Hey @hkirat , are you also interested in integrating crypto payments with coinbase commerce api? If yes, I can start working on that. |
65c1ce5
to
dc6d48a
Compare
Hey @hkirat, are you still interested in this feature? |
yes this is top priority for me to review |
So the signup logic here is useful, I'll be copying that over I'll be shelving the PR for now (not closing it as we may need payments in the future). But we will have to get rid of the client side call here - and replace it with a webhook from rzp - https://github.com/code100x/cms/pull/324/files#diff-68bf6bcd40c0526800dac7b625b2c2ecaee8433b54f07d37738e3155cd1fe501R55 |
/bounty $25 for the effort |
7619534
to
da7f50d
Compare
28c3c52
to
a1b63ce
Compare
Hi @hkirat , I have added a webhook for razorpay. You can test this webhook on localhost with |
@dhruvilmehta will be testing it ASAP |
Hi @siinghd , are you testing it? If everything seems fine, we can merge it before Web3 cohort launch. Just the UI part would be remaining that can be done ASAP. |
PR Fixes:
Resolves #169
Checklist before requesting a review
Similar to @devsargam but added a little tweaks to DB schema and URLs for buying courses.
UI is broken for now but can be done soon once this is approved.
Approach:
DB Schema
Flow of payment
Note: 8 files just have formatting changes due to prettier check while committing.